Skip to content

Run TypeScript SDK unit tests on pull requests - #74513

Merged
jason810496 merged 4 commits into
apache:mainfrom
kevinhongzl:run-ts-sdk-unit-tests-in-ci
Oct 12, 2026
Merged

jason810496 merged 4 commits into
apache:mainfrom
kevinhongzl:run-ts-sdk-unit-tests-in-ci

Conversation

@kevinhongzl

@kevinhongzl kevinhongzl commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

closes: #74061

This runs the TS SDK unit tests on PRs that change ts-sdk/, as already happens for Go and Java.


Was generative AI tooling used to co-author this PR?
  • Yes - Claude Code (Opus 5.5)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

@Andrushika Andrushika left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!
Nit: could we exclude ts-sdk/docs/ to avoid docs changes triggering the unit tests?

@kevinhongzl

Copy link
Copy Markdown
Contributor Author

Thanks! Nit: could we exclude ts-sdk/docs/ to avoid docs changes triggering the unit tests?

Sure! Appreciate your review :)

@kevinhongzl
kevinhongzl force-pushed the run-ts-sdk-unit-tests-in-ci branch from 3dfc1a5 to d3ca2d1 Compare October 10, 2026 03:33
@kevinhongzl
kevinhongzl force-pushed the run-ts-sdk-unit-tests-in-ci branch from d3ca2d1 to f02d114 Compare October 10, 2026 09:12

@jason810496 jason810496 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thanks.

@jason810496
jason810496 merged commit 270b2ab into apache:main Oct 12, 2026
161 checks passed
@kevinhongzl
kevinhongzl deleted the run-ts-sdk-unit-tests-in-ci branch October 13, 2026 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TS SDK unit tests do not run in pull request CI

3 participants